Skip to content

Parse Range header values. - #112

Merged
ioquatix merged 1 commit into
mainfrom
add-range-header
Aug 2, 2026
Merged

Parse Range header values.#112
ioquatix merged 1 commit into
mainfrom
add-range-header

Conversation

@samuel-williams-shopify

@samuel-williams-shopify samuel-williams-shopify commented Aug 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Add Protocol::HTTP::Header::Range with support for byte and extension range units.
  • Resolve bounded, open-ended, and suffix byte ranges against a representation size.
  • Parse Range through the default Protocol::HTTP::Headers policy.
  • Add an unreleased release note.

Tests

  • bundle exec bake test (760 tests, 1,537 assertions, 100% coverage).
  • bundle exec bake decode:index:coverage lib (438/438 definitions).
  • Focused RuboCop passes.

Contribution

  • I added tests for my changes.
  • I tested my changes locally.
  • I agree to the Developer Certificate of Origin 1.1.

@samuel-williams-shopify
samuel-williams-shopify force-pushed the add-range-header branch 3 times, most recently from 8736609 to 51dfa71 Compare August 2, 2026 11:39
@samuel-williams-shopify samuel-williams-shopify mentioned this pull request Aug 2, 2026
Open
3 tasks
@ioquatix
ioquatix merged commit 834c452 into main Aug 2, 2026
34 of 40 checks passed
@ioquatix
ioquatix deleted the add-range-header branch August 2, 2026 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@samuel-williams-shopify @ioquatix